Skip to content
This repository has been archived by the owner on Jun 25, 2024. It is now read-only.

Drop DataPlaneService ConfigMaps and Secrets #923

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

slagle
Copy link
Collaborator

@slagle slagle commented Jun 6, 2024

  • Drops the ConfigMaps and Secrets fields from OpenStackDataPlaneService.
  • Updates all services under config/services to use the
    OpenStackDataPlaneService.Spec.DataSources field instead.
  • Removes some un-needed labels from some services as they aren't needed
    and for brevity.
  • Update docs for the switch to DataSources.

Depends-On: openstack-k8s-operators/install_yamls#846
Depends-On: openstack-k8s-operators/ci-framework#1847
Depends-On: openstack-k8s-operators/architecture#276
Signed-off-by: James Slagle [email protected]

Copy link
Contributor

openshift-ci bot commented Jun 6, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: slagle

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci bot added the approved label Jun 6, 2024
@slagle slagle force-pushed the service-datasources branch from 1f62f0b to 7dda707 Compare June 6, 2024 18:50
@slagle
Copy link
Collaborator Author

slagle commented Jun 6, 2024

/retest

Copying blob sha256:793a8691df4a71168cb733364beb6d4e24022b9af46f047495a426c932a8f52e, the build src-amd64 failed after 22s with reason OutOfMemoryKilled: The build pod was killed due to an out of memory condition.

@slagle slagle force-pushed the service-datasources branch from 7dda707 to 2b8dd80 Compare June 6, 2024 20:54
- Drops the ConfigMaps and Secrets fields from OpenStackDataPlaneService.
- Updates all services under config/services to use the
  OpenStackDataPlaneService.Spec.DataSources field instead.
- Removes some un-needed labels from some services as they aren't needed
  and for brevity.
- Update docs for the switch to DataSources.

Depends-On: openstack-k8s-operators/install_yamls#846
Depends-On: openstack-k8s-operators/ci-framework#1847
Depends-On: openstack-k8s-operators/architecture#276
Signed-off-by: James Slagle <[email protected]>
@slagle slagle force-pushed the service-datasources branch from 2b8dd80 to 6f354e2 Compare June 6, 2024 21:19
@slagle
Copy link
Collaborator Author

slagle commented Jun 6, 2024

/retest

ERRO[2024-06-06T21:22:07Z] Some steps failed: ERRO[2024-06-06T21:22:07Z] could not run steps: step src failed: error occurred handling build src-amd64: could not get build src-amd64: builds.build.openshift.io "src-amd64" not found

@slagle
Copy link
Collaborator Author

slagle commented Jun 6, 2024

/retest

2 similar comments
@slagle
Copy link
Collaborator Author

slagle commented Jun 6, 2024

/retest

@lewisdenny
Copy link

/retest

Copy link

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://review.rdoproject.org/zuul/buildset/46856b5c6643467e99278a6a3156de87

✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 02m 32s
podified-multinode-edpm-deployment-crc FAILURE in 1h 37m 32s
cifmw-crc-podified-edpm-baremetal FAILURE in 1h 30m 05s
✔️ dataplane-operator-docs-preview SUCCESS in 2m 28s

@slagle
Copy link
Collaborator Author

slagle commented Jun 6, 2024

/retest

@fao89
Copy link
Collaborator

fao89 commented Jun 7, 2024

/retest-required

@fao89
Copy link
Collaborator

fao89 commented Jun 7, 2024

recheck

Copy link
Contributor

openshift-ci bot commented Jun 7, 2024

@slagle: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/dataplane-operator-build-deploy-kuttl 6f354e2 link true /test dataplane-operator-build-deploy-kuttl
ci/prow/functional 6f354e2 link true /test functional

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@slagle
Copy link
Collaborator Author

slagle commented Jun 7, 2024

I don't think this will pass until openstack-operator is bumped to pick up the new runner image.

Copy link

This change depends on a change that failed to merge.

Change openstack-k8s-operators/architecture#276 is needed.

@openshift-merge-robot
Copy link
Contributor

PR needs rebase.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

Copy link

@bogdando bogdando left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this should wait for the adoption job to CI such sort of changes perhaps (as this will break adoption)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants